Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(ce): cost anomaly monitor and subscription #28547

Closed
wants to merge 2 commits into from

Conversation

jogold
Copy link
Contributor

@jogold jogold commented Jan 1, 2024

Add L2 constructs for cost anomaly detection.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Add L2 constructs for cost anomaly detection.
@github-actions github-actions bot added the p2 label Jan 1, 2024
@aws-cdk-automation aws-cdk-automation requested a review from a team January 1, 2024 16:38
@github-actions github-actions bot added the distinguished-contributor [Pilot] contributed 50+ PRs to the CDK label Jan 1, 2024
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 2525154
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Jan 2, 2024
@kaizencc
Copy link
Contributor

kaizencc commented Jan 8, 2024

Hi @jogold, thanks for your contribution -- under our new contribution policy for new L2 alpha constructs, we want to see additional signal from the community before we invest in reviewing, maintaining, and eventually stabilizing this.

There's a new section in our contribution guide that outlines demonstrating value, and specifically the best way to demonstrate value here is to have an existing module with [some] userbase. This will signal to us that both the API is ergonomic and that there is a real use case for higher level CDK constructs for the resource.

In that vein, I encourage you to self-publish this as a new module and have it listed on construct hub. From there, we can monitor usage and see if there is value to include within cdk. I am closing this PR because it is not something that we are ready to accept today. I hope you understand.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distinguished-contributor [Pilot] contributed 50+ PRs to the CDK p2 pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants